Skip to content

Clarify migration steps for overviews table issue#787

Open
JaroslavMasar74 wants to merge 3 commits into
mainfrom
fix_SQL_table_migration_mm
Open

Clarify migration steps for overviews table issue#787
JaroslavMasar74 wants to merge 3 commits into
mainfrom
fix_SQL_table_migration_mm

Conversation

@JaroslavMasar74

Copy link
Copy Markdown
Contributor

Updated migration instructions to clarify the fix for the missing overviews table during older database migrations.

Updated migration instructions to clarify the fix for the missing overviews table during older database migrations.

@JaroslavMasar74 JaroslavMasar74 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

edited according to comments

Comment thread src/server/upgrade/index.md Outdated
```bash
select * from project;
```
```bash

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could probably just add link to public repository sql.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes I can only upload the link to download the SQL file

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we discussed that and for convenience I would keep it in docs, that's the source where admins are copy pasting commands

Perform the migration:

0. Fix table - migration from old to new MM version
0. Fix the issue of missing overviews table in case it was not created during older db migration.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this could go to the step where migrations are running - no step 0. You need to double check if map_overviews table is there and if not just run sql.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it would be a good idea to fix this before running the migration so that there is no error message about the missing table.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this table fix belongs to 2025.7.3 migration as that is the one which relies on that missing table (and potentially also newer ones).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants